Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[pull] main from geoserver:main #40

Merged
merged 63 commits into from
Nov 29, 2024
Merged

[pull] main from geoserver:main #40

merged 63 commits into from
Nov 29, 2024

Conversation

pull[bot]
Copy link

@pull pull bot commented Nov 15, 2024

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

jodygarnett and others added 24 commits October 29, 2024 19:13
The docuemntation was restructured to seperate out administrator from worksapce administrator in response to security vulnerability on log file traversal.
#7961)

fall back to using the URL of the capabilities document itself.

Update MapMLBaseProxyTest.java to check the cascaded request against the
request URL from the capabilities document OR if that is null, use the
URL OF the capabilities document as the base.

Update assertCascading in MapMLWMTSProxyTest.java to compare against
resource request URL from capabilities document OR if that is null,
use the URL OF the capabilities document as the base.
…ment section

* Centralize the declaration of geotools dependencies to the root pom's
  dependencyManagement and remove `<version>${gt.version}</version>` in
  module poms for consistency.

* Exclude `xml-apis` and apache `fop` in the root pom dependency management section
  Of especial interes for the `gt-app-schema` module not to carry over the
  unnecessary (sice Java 1.6) `xml-apis` jar, whose presence also makes the
  eclipse IDE complain there are multiple sources for classes
  in the `javax.xml.namespace`, `org.w3c.dom`, and other packages.

commit 397981a4e9dc310219270f82b2e560f6d3de4127
Author: Gabriel Roldan <[email protected]>
Date:   Tue Aug 6 12:05:38 2024 -0300
…rom_root_pom

Declare all GeoTools dependencies in the root pom's dependency management section
The way it is stated now is misleading, because the LEGEND_OPTIONS params are required, but they are mentioned after the optional parameters.
Add a data directory under build/cite/ogcapi-features10.

This is an extract of the [release](../../../data/release/) data directory
without the raster layers and their corresponding datasets, for the purpose
of executing the ogcapi-features-1.0 conformance test suite against this
well known datasets.

Note although it adds the shapefiles and geopackage again, they won't incur
in any size overhead for the repository given the way git stores files.
This commit adds support for running CITE (Compliance & Interoperability Testing Evaluation)
tests for `OGC API - Features 1.0`. It includes updates to dependencies, configuration changes,
a new CI workflow, and test data handling to ensure proper validation of GeoServer¿s conformance.

* The new `ogcapi-features10` test suite runs against the latest version of the
  [ogccite/ets-ogcapi-features10:1.7.1-teamengine-5.4.1](https://hub.docker.com/r/ogccite/ets-ogcapi-features10/tags)
   "TEAM Engine and OGC API - Features test suite", using the TeamEngine¿s REST API. Refer to the "Using the REST API"
   section in the [User Guide](https://opengeospatial.github.io/teamengine/users.html) for more details.
* Introduced a new GitHub Actions workflow that builds and runs the CITE tests.
* The CITE tests run against all feature types present in the `release` data directory.
    * Configured the WFS to declare `EPSG:4326` and `EPSG:3857` for all FeatureTypes,
      ensuring that tests do not attempt reprojection between incompatible coordinate systems or boundaries.
    * Disabled the `tiger:giant_polygon` FeatureType as the test suite tries to reproject it
      to CRSs that can't cover its single world-wide polygon.
* Updated the CITE build and run scripts located in `build/cite/` to ensure they can be executed.
    * The existing CITE tests continue to run via CLI commands against the `geoserver-docker.osgeo.org/geoserver-cite:teamengine_latest` image.
      to include only the necessary dependencies for this specific test suite.
    * Updated the `geoserver.war` file used for testing to default to the
      locally built version. A `make war` target has been introduced.
    * Added an `ogcapi-features` Maven profile in `src/web/app/pom.xml`
@pull pull bot added the ⤵️ pull label Nov 15, 2024
`EPSG:4087` (WGS 84 / World Equidistant Cylindrical) is a global
projection, while `EPSG:3857` (WGS 84 / Pseudo-Mercator) is not.

Since the `ogcapi-features-1.0` ETS fails while doing coordinate
transformations on itself, despite GeoServer having returned
transformed geometries (albeit outside the area of validity of
the target CRS), let's use two default published CRS's that
cover the whole globe.

Re-enable the `tiger:giant_polygon` layer, since now it can be
reprojected across the declared global CRS's.
…imitations

The ets-ogcapi-features10 test suite will chek the transformed collection items
fall inside the collection's declared spatialExtent (WGS84). In doing so, the test
themselves are limited to 7 decimals for geographic coordinates, and hence will fail
with GeoServer's higher accuracy, despite GeoServer responses being correct.
…nal fixes

Use a patched version of `ogccite/ets-ogcapi-features10:1.8-SNAPSHOT-teamengine-5.4.1`.

Currently, we require the following pull request to be applied:
[Fix global crs codes lookup in discovery collections crs tests](opengeospatial/ets-ogcapi-features10#255)
to the test suite.

This may happen again, hence we're introducing a build step to generate
the required `ogccite/ets-ogcapi-features10` image out of the
`geoserver/ets-ogcapi-features10` fork, using a branch named `geoserver/integration`.

The procedure to maintain that branch is to rebase on top of the upstream
master branch as required, and apply the additional branches with `git merge --no-ff`.
Use a patched ogccite/ets-ogcapi-features10 Docker image with additional fixes
mprins and others added 29 commits November 18, 2024 15:46
Missing end quote '
Capitalised JSON
[GEOS-11603] KML download mode does not show layer titles
…st A Slash Before the Question Marks Causes Issues

Removed unnecessary question mark stuff
Add additional component check
Switch to Discourse developer group from email list
* [GEOS-11616] GSIP 229 - File system access isolation

* [GEOS-11616] GSIP 229 - File system access isolation. Windows compatibility
As this was a breaking change it should be a little bit more clear  about URL Checks.
Revert to build the
`ogccite/ets-ogcapi-features10:1.8-SNAPSHOT-teamengine-5.4.1` docker
image for ogcapi-features CITE testing from the official repository's
`master` branch now that the [requried fix](opengeospatial/ets-ogcapi-features10#255)
is merged.
…ures10-master

ogcapi-features CITE tests: build from official git repository
Using the latest tag for the ogcapi-features teamengine docker
image, as built by the custom script, which tags both latest and
1.8-SNAPSHOT-teamengine-<version>, where 1.8 and <version> are moving
targets.
* APIConformance outline
* Outline ConformanceAdminPanel
* Update APIConformance with OGC Standard Level language
  This repalce status and provides concepts such as COMMUNITY_STANDARD and RETIRED_STANDARD
* Working AdminPagePanel for FeatureConformance, ECQLConformance and CQL2Conformance
* Add CQL2 to FeatureServiceAdminPanel
* Work on load/save
* Change to java beans configuration
* Save feature service configuration beans
* Visual feedback for conformance null / null / true / false state
* Use conformance to check parameters, provide warning when ignored
* Document new configuration options
* Fix checkstyle violation
* Revise based on feedabck and discussion
* Double check GMLSF2 is disabled to confirm CITE tests should pass
* Connect search, queryable and functions
* Adress feedback, fix headers, remove commeneded out code, remove duplication etc...
* Headers
* Use a three-state checkbox to preserve conformance default value
  The checkboxes in FeatureServiceAdminPanel would otherwise update
  the model value to false when it was originally null.

  This way the XStream persister will get the unchanged values as null
  instead of false.

* Address QA and formatting warnings

---------

Co-authored-by: Gabriel Roldan <[email protected]>
@pull pull bot merged commit 3a8c52c into GeoCat:main Nov 29, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.